Skip to content

windows: improve/fix raw mode #815

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Atrox
Copy link

@Atrox Atrox commented Aug 27, 2023

Raw mode didn't quite work for me as I would have expected on windows. For example: function and arrow keys didn't work at all.

I took inspiration from the raw console mode set in the containerd/console library. After setting those verbatim, everything worked as expected on my side.

Please take those changes with a grain of salt, I'm not a windows guy.

@Atrox Atrox requested a review from TimonPost as a code owner August 27, 2023 23:01
@Atrox Atrox force-pushed the bugfix/win-raw-mode branch from 85d00f0 to 1c7145d Compare August 27, 2023 23:06
@Atrox Atrox force-pushed the bugfix/win-raw-mode branch from 1c7145d to 7d4aaa9 Compare August 27, 2023 23:07
@sigmaSd
Copy link
Contributor

sigmaSd commented Aug 27, 2023

This is the first time I see an issue about this, and I used crossterm on windwos for years now with no issues

I think its better if you open an issue first with what you encountered (+reproduction + os version) before attempting changes with no clear benefit and that in-itself isn't clear

@Atrox
Copy link
Author

Atrox commented Aug 28, 2023

To reproduce my issue on windows: Enable raw mode with crossterm::terminal::enable_raw_mode(), press arrow up on the keyboard. Arrow up is not getting sent to the application and instead processed by the terminal/windows.

@lcmgh
Copy link

lcmgh commented Oct 18, 2023

Might this also fix #768 ? @Atrox

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants